Skip to content

fix(ci): remove redundant git-cliff output config to fix publish-release#619

Merged
olivermeyer merged 1 commit into
release/v1.3.0from
fix/publish-release
May 4, 2026
Merged

fix(ci): remove redundant git-cliff output config to fix publish-release#619
olivermeyer merged 1 commit into
release/v1.3.0from
fix/publish-release

Conversation

@olivermeyer

Copy link
Copy Markdown
Collaborator

Why?
The publish-release workflow was failing with git-cliff error: '-o' and '-p' can only be used together if they point to different files. The [tool.git-cliff.changelog] section in pyproject.toml sets output = "CHANGELOG.md" (implicitly passing -o), while the workflow passes --prepend CHANGELOG.md (-p) explicitly — both pointing to the same file, which git-cliff now rejects.

How?
Removed the output = "CHANGELOG.md" line from [tool.git-cliff.changelog] in pyproject.toml. The workflow already controls the output path explicitly via --prepend and --output flags in each branch of the conditional, so the config-level default is redundant.

Copilot AI review requested due to automatic review settings May 4, 2026 07:37
@sonarqubecloud

sonarqubecloud Bot commented May 4, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the release automation configuration by removing a redundant git-cliff changelog output setting from pyproject.toml. In the SDK’s release process, publish-release.yml already decides whether changelog content is prepended to or written into CHANGELOG.md, so removing the config-level default avoids the conflicting -o/-p combination that was breaking release publication.

Changes:

  • Remove output = "CHANGELOG.md" from [tool.git-cliff.changelog].
  • Leave changelog file selection to the release workflows, which already pass --prepend or --output explicitly.

@olivermeyer olivermeyer added the skip:test:long_running Skip long-running tests (≥5min) label May 4, 2026
@olivermeyer olivermeyer enabled auto-merge (rebase) May 4, 2026 07:52
@olivermeyer olivermeyer merged commit a97ccb9 into release/v1.3.0 May 4, 2026
27 checks passed
@olivermeyer olivermeyer deleted the fix/publish-release branch May 4, 2026 08:07
@codecov

codecov Bot commented May 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.
see 5 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip:test:long_running Skip long-running tests (≥5min)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants